Conversation
Plain Craft Launcher 2/Pages/PageInstance/PageInstanceSetup.xaml
Outdated
Show resolved
Hide resolved
Signed-off-by: Harry-hq <dq114144@126.com>
DotnetInstall
left a comment
There was a problem hiding this comment.
必须修改
1.tokenizer 仍有缺陷:当前双引号解析在引号不配对时会静默出错,用户毫无感知。更根本的问题是,环境变量值本身可以含换行符,空格分隔方案从设计上就不健全。建议改用换行分隔(TextBox 设为多行),或按 ruattd 的建议使用 JsonSerializer 存储,彻底规避歧义。
2.PATH 覆盖风险:_ApplyInstanceEnvVariables 在启动器已设好 Path 之后执行,用户若填写 Path=... 会直接覆盖掉 PCL 拼好的 Java 路径,导致游戏启动失败且报错完全不直观。至少需要对 Path/PATH 做特殊处理(拒绝覆盖或改为追加),并在 Tooltip 中明确警告。
3.Tooltip 示例有误导性:示例 JAVA_HOME=C:\Java 没有空格,但真实场景几乎都是 C:\Program Files\Java\jdk-21。应在示例中展示含空格路径的正确写法(加双引号),否则用户遇到含空格路径时会被静默截断。
应改进
4.缺少注入日志:环境变量注入成功与否在启动日志中完全不可见,出问题时排查困难,建议至少记录注入了哪些 key(值可脱敏)。
5.Key 未做合法性校验:若用户误输入导致 key 含空格等非法字符,会被静默注入,建议加 Trim() 并跳过含空格的 key。
现在遇到了问题:日志系统不支持输出换行字符串 |
|
根据社区开发者的讨论,我们仍然无法接受你现在 vibe coding 的结果,抱歉 |
抱歉第一次 PR 眼睛瞎了没看见这寄吧 AI 瞎改 #2490
